Skip to content

add hidden fields facility for booking form#5363

Merged
VladMstv merged 2 commits intorelease/1.22from
ssn-config
Dec 3, 2025
Merged

add hidden fields facility for booking form#5363
VladMstv merged 2 commits intorelease/1.22from
ssn-config

Conversation

@imbenham
Copy link
Contributor

@imbenham imbenham commented Dec 3, 2025

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a hidden fields facility for the booking form, allowing specific fields to be conditionally hidden based on configuration. The main changes involve refactoring the prepopulateBookingForm function to use the new hiddenBookingFields configuration and improving code organization.

Key Changes:

  • Exported prepopulateBookingForm as a standalone function instead of part of BOOKING_CONFIG
  • Added hiddenBookingFields array with type safety via BookingQuestionnaireLinkId type
  • Modified SSN field visibility logic to respect the hiddenBookingFields configuration

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
packages/zambdas/src/patient/paperwork/get-booking-questionnaire/index.ts Updated import to use prepopulateBookingForm directly instead of accessing it via BOOKING_CONFIG
packages/utils/lib/configuration/booking/index.ts Refactored booking configuration: moved prepopulateBookingForm to be an exported function, added hiddenBookingFields facility with type-safe array, and updated SSN field logic to check hidden fields configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@VladMstv VladMstv merged commit da82cc1 into release/1.22 Dec 3, 2025
8 of 10 checks passed
@VladMstv VladMstv deleted the ssn-config branch December 3, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants